Remove findbugs from Travis and simplify the yml configuration file #2620
Conversation
The findbugs:findbugs Maven phase was added to the Travis test script as a way to flag errors. However this is generated a report while the command to fail the build is findbugs:check. The latter generates many warnings across components which cannot be fixed in the main Bio-Formats source code without significant work. Instead these should be fixed and activated in each decoupled component. This commit removes `findbugs:findbugs:` from the Travis test and simplifies the YML file to clarify the fact we are testing a 2x2 matrix (Ant/Maven with JDK 7/8).
|
Should we create a card for coordinating a findbugs cleanup effort with activating |
|
@mtbc given the output of the command, my impression is certainly that activating |
|
The changes look fine and travis is green. Please merge. Seconded on scheduling findbugs cleanup in the split components. Doing it bioformats in due course would also be desirable if possible. |
The
findbugs:findbugsMaven phase was added to the Travis test script as a way to flag errors. However this is generated a report while the command to fail the build isfindbugs:check. The latter generates many warnings across components which cannot be fixed in the main Bio-Formats source code without significant work. Instead these should be fixed and activated in each decoupled component.This commit removes
findbugs:findbugs:from the Travis test and simplifies the YML file to clarify the fact we are testing a 2x2 matrix (Ant/Maven with JDK 7/8).To test this PR, check Travis is green and the time of
oraclejdk8/mavencomponent is reduced to match roughly the time of theopenjdk8/mavencomponent.